Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI: bump GHC to recommended version (9.4.7) #1075

Merged
merged 1 commit into from
Oct 30, 2023
Merged

Conversation

andreasabel
Copy link
Member

No description provided.

@felixwellen
Copy link
Collaborator

Looks reasonable to me -> merging.

@felixwellen felixwellen merged commit b822ba9 into master Oct 30, 2023
1 check passed
@andreasabel andreasabel added the re: CI Concerning continuous integration label Oct 30, 2023
@andreasabel andreasabel deleted the bump-ci-ghc branch October 30, 2023 20:59
@felixwellen
Copy link
Collaborator

I looks a lot like this commit pushed the checking time for the library up from around 35min to 50min. Is there a reason for that?

@felixwellen
Copy link
Collaborator

Might be just noise in the github-builttimes...

@andreasabel
Copy link
Member Author

Looking at recent runs, I see build times between 3 and 110 minutes: https://github.com/agda/cubical/actions?query=branch%3Amaster+event%3Apush
What seems to matter most here is how much of the cache can be reused.

A priori, I do not see a reason why bumping the GHC version would cause a regression in Agda type-checking times, but of course there could be a GHC regression... One would need some reproducer...

@felixwellen
Copy link
Collaborator

Thanks for the input - I guess I should do some profiling locally, when I have some time...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
re: CI Concerning continuous integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants